Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open WebUI App for Dashboard #3869

Open
wants to merge 15 commits into
base: development
Choose a base branch
from
Open

Open WebUI App for Dashboard #3869

wants to merge 15 commits into from

Conversation

mik-tf
Copy link
Contributor

@mik-tf mik-tf commented Feb 6, 2025

Description

Specs

  • We don't need ipv4
  • User can deploy with gateway domain or custom domain
  • Only works for CPU now
    • Later we can do a version for GPU

General Checklist

Notes

Copy link
Contributor

@amiraabouhadid amiraabouhadid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i selected node 156, but it keeps deploying on node 313 which fails
https://www.loom.com/share/625d1cd017cd40fda4cb3fe0d800ed75?sid=024f700b-dd09-433b-b3e9-f981ece2d183

packages/playground/src/weblets/tf_openwebui.vue Outdated Show resolved Hide resolved
packages/playground/src/weblets/tf_openwebui.vue Outdated Show resolved Hide resolved
packages/playground/src/weblets/tf_openwebui.vue Outdated Show resolved Hide resolved
packages/playground/src/weblets/tf_openwebui.vue Outdated Show resolved Hide resolved
packages/playground/src/weblets/tf_openwebui.vue Outdated Show resolved Hide resolved
@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 10, 2025

i selected node 156, but it keeps deploying on node 313 which fails https://www.loom.com/share/625d1cd017cd40fda4cb3fe0d800ed75?sid=024f700b-dd09-433b-b3e9-f981ece2d183

Thanks for testing. I can't reproduce on my end. Deployed on node 156 devnet:

image

As I understand the issue your encountered is with the gateway, not the deployment per se.

@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 10, 2025

@amiraabouhadid @maayarosama thanks for the review. Lmk if anything else is needed.

@maayarosama
Copy link
Contributor

Works fine
Screenshot from 2025-02-10 12-09-06

@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 12, 2025

Thanks @maayarosama for the review + approval. One more to go!

Lmk anyone if anything else is needed.

@mik-tf mik-tf marked this pull request as draft February 13, 2025 17:14
@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 16, 2025

Update

  • I started working directly on the GPU version
  • The flist full vm gpu version is done and approved
  • I am finalizing the dashboard app (only need to add the gateway domain section and I will make the PR as ready for review)

@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 17, 2025

Update

  • We decided to publish right away the GPU version of the app, instead of CPU only
  • For this we needed to use a full VM instead (the flist full vm version works 100% and is already accepted in tf-images repo)
  • WIP (see below)
  • Note: I will get rid of unused variables when I find the gateway fix (failed Lint test)

TODO

  • All that is left to do is to finalize the logic for the gateway domain (with port 8080) and custom domain (with port 8080). This means also adding the envs variable OPENWEBUI_DOMAIN.
  • Ideally we can present this at ETH Denver. I think I can find the fix on my own, but if devs have any time to check on Monday, it would be VERY appreciated as I am sure you guys can know quickly what I am missing. @xmonader. That being said, I can find it with enough time on my side. Thanks!

@mik-tf mik-tf marked this pull request as ready for review February 18, 2025 01:12
@mik-tf
Copy link
Contributor Author

mik-tf commented Feb 18, 2025

Update

The PR is ready for review with GPU version.

Copy link
Contributor

@amiraabouhadid amiraabouhadid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please merge development into this branch to avoid having wireguard network only selected but mycelium only domains showing up in the domains selection
otherwise, you'll end up with this err (i think)
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dashboard Solution: Open WebUI
4 participants